Skip to content

Conversation

@IgorBenko
Copy link
Contributor

A PR for the menu entry will be created separately.

@IgorBenko IgorBenko requested a review from a team December 9, 2025 14:19
@IgorBenko IgorBenko requested a review from pscheit December 15, 2025 08:28
@pscheit pscheit merged commit af62677 into main Dec 15, 2025
@pscheit pscheit deleted the package-metadata branch December 15, 2025 13:29
CURLOPT_RETURNTRANSFER => true,
CURLOPT_HTTPAUTH => CURLAUTH_BASIC,
CURLOPT_USERPWD => 'token:' . $token
]);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if we want to spend more time on this, but generally prefer to teach people good ideas in docs, e.g. using symfony http client rather than plain curl? But guess enough people will want to use this 🤦

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah sorry ignore me. Really got more important things to do sorry for the distraction, we can revisit this at another point and hopefully people work this out themselves :D

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, that would be nicer, but I guess non-symfony devs might prefer seeing something more general :)
Maybe switching the example to guzzle would be a good compromise?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants